Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REWRITE #77

Draft
wants to merge 31 commits into
base: main
Choose a base branch
from
Draft

REWRITE #77

wants to merge 31 commits into from

Conversation

not-nef
Copy link
Member

@not-nef not-nef commented Aug 30, 2023

complete rewrite from the ground up with oop and good this time

help much appreciated

why is the file diff so ducked up

@not-nef not-nef self-assigned this Aug 30, 2023
@sumeshir26
Copy link
Member

Whats the point of the StartupWindow?

@not-nef
Copy link
Member Author

not-nef commented Aug 31, 2023

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

@not-nef
Copy link
Member Author

not-nef commented Aug 31, 2023

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

Also i will add things like recently opened files there later

@not-nef
Copy link
Member Author

not-nef commented Aug 31, 2023

you guys were so right oop is so cool

@not-nef
Copy link
Member Author

not-nef commented Sep 1, 2023

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

Also i will add things like recently opened files there later

nevermind i got rid of it

@im-coder-lg
Copy link
Member

@not-nef need your insight on this review! #77 (comment)

@not-nef
Copy link
Member Author

not-nef commented Sep 1, 2023

idk what you want me to look at, that hyperlink does nothing for me

@im-coder-lg
Copy link
Member

@not-nef I meant this in that link:

You can find more information in the [wiki](https://github.com/not-nef/onyx/wiki)

Aren't we supposed to change that?

@not-nef
Copy link
Member Author

not-nef commented Sep 3, 2023

yeah i will change that in this pr, a lot needs to be redone in this repo

@not-nef
Copy link
Member Author

not-nef commented Sep 3, 2023

most docs files are out of date

@not-nef
Copy link
Member Author

not-nef commented Sep 7, 2023

linux is really starting to grow on me but i cant switch because i wont be able to play most of my video games anymore :(

@not-nef
Copy link
Member Author

not-nef commented Sep 7, 2023

you know what, im just gonna make a seperate windows partition for my games and switch to linux.

@not-nef
Copy link
Member Author

not-nef commented Sep 7, 2023

my new motherboard should be arriving today so i am very excited.

@im-coder-lg
Copy link
Member

We gotta update the branch.

@im-coder-lg
Copy link
Member

@not-nef you added TkFileBrowser to the code. It's not there in requirements.txt yet, maybe?

@not-nef
Copy link
Member Author

not-nef commented Sep 30, 2023

yes, because i will most likely add more packages and will update requirements.txt before merging the branch

@im-coder-lg
Copy link
Member

Okay, so what about a Gitpod config? I need to add it just for configuring the image to load with VNC preview enabled. It uses Docker, so there would be an issue with that.

@im-coder-lg
Copy link
Member

We wouldn't need it much, yes, but if anybody were to test on the fly, it might help. I'm not coding anything on my local computer since Zorin and Python seem to be at odds right now.

@not-nef
Copy link
Member Author

not-nef commented Sep 30, 2023

so the app is lagging like crazy, but the tkfilebrowser instance inside of it is running smooth as ever. something is clearly not right

@im-coder-lg
Copy link
Member

So, it isn't working on Gitpod. Asciinema(basically fancy terminal video) clip:
asciicast

@not-nef
Copy link
Member Author

not-nef commented Sep 30, 2023

manually set TZ=UTC in your environment

source

@im-coder-lg
Copy link
Member

Will do so later, thanks!

@im-coder-lg
Copy link
Member

im-coder-lg commented Oct 1, 2023

It seems to be rapid for me, apart from the lag you get from noVNC. @not-nef try this, exactly as I say.
Make a new folder, clone PR #77 there, and just initialise a virtualenv there. I think we must start this from the start to test it. Install the modules, and install tkfilebrowser separately. Then, try running the app from src from that folder. Maybe that could tell us if it is a module or your system, coz on Gitpod, it's dandy, as they say.

@im-coder-lg
Copy link
Member

Save functions aren't working though, maybe a bug. How do we save it in the first place?

@not-nef
Copy link
Member Author

not-nef commented Oct 2, 2023

It seems to be rapid for me, apart from the lag you get from noVNC. @not-nef try this, exactly as I say.

Make a new folder, clone PR #77 there, and just initialise a virtualenv there. I think we must start this from the start to test it. Install the modules, and install tkfilebrowser separately. Then, try running the app from src from that folder. Maybe that could tell us if it is a module or your system, coz on Gitpod, it's dandy, as they say.

You think its the filebrowser module?

@not-nef
Copy link
Member Author

not-nef commented Oct 2, 2023

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

@im-coder-lg
Copy link
Member

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

Usually, we have a save button, or at least a shortcut. Neither exists in that, as well as I can't go through folders yet.

You think its the filebrowser module?

Well, that gives a save issue for me, but I think it could be some module that just keeps reloading or something, but is building up the data, piling it up. Maybe that could be the reason behind the lag? I don't know.

@not-nef
Copy link
Member Author

not-nef commented Oct 2, 2023

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

Usually, we have a save button, or at least a shortcut. Neither exists in that, as well as I can't go through folders yet.

It is in the menu at the top of the screen

You think its the filebrowser module?

Well, that gives a save issue for me, but I think it could be some module that just keeps reloading or something, but is building up the data, piling it up. Maybe that could be the reason behind the lag? I don't know.

i need to further investigate

@im-coder-lg
Copy link
Member

There isn't a save button in the Save As UI. Like the button we have on Windows for saving. It could also be due to Xfce.

@not-nef
Copy link
Member Author

not-nef commented Oct 8, 2023

finally found some time to code again. im definitely not bugging. i got the main branch to sort of run and it is blazing fast as usual. that means theres got to be something wrong with my code. time to dig

@not-nef
Copy link
Member Author

not-nef commented Oct 8, 2023

i am 99% sure its an issue with sv_ttk because if i comment out the line that sets the theme, everything is fast as usual

@im-coder-lg
Copy link
Member

I think you might have kept something weird with the theme part. Check that out, it might give some insight, since that could easily cause lag.

@not-nef
Copy link
Member Author

not-nef commented Oct 8, 2023

i dont know what you mean

@im-coder-lg
Copy link
Member

I didn't explain. Maybe you set up an auto-detection for the system theme, but it kinda loops over and over again, making new processes I guess. Could be something similar.:shrug:

@not-nef
Copy link
Member Author

not-nef commented Oct 8, 2023

i didnt set up theme detection yet

@im-coder-lg
Copy link
Member

I thought you did. My bad, I got confused. Maybe the theme code is sort of just running on a loop?

@im-coder-lg
Copy link
Member

I know it's selective, but maybe it is running that continuously.

@rdbende
Copy link

rdbende commented Oct 21, 2023

i am 99% sure its an issue with sv_ttk because if i comment out the line that sets the theme, everything is fast as usual

sv_ttk is slow on Windows

@rdbende
Copy link

rdbende commented Oct 21, 2023

@not-nef
Copy link
Member Author

not-nef commented Oct 21, 2023

but why is the version before the rewrite so fast?

@rdbende
Copy link

rdbende commented Oct 21, 2023

Yeah, that's strange. Are you using a type of widget in the new version that you weren't before?

@not-nef
Copy link
Member Author

not-nef commented Oct 21, 2023

not that i could think of, but everything is oop now

@rdbende
Copy link

rdbende commented Oct 21, 2023

but everything is oop now

That shouldn't cause slowness

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants